Skip to content

add function to preprocess for phase data#63

Merged
kenjiro-mk merged 9 commits intomasterfrom
dev_kuroda
Jan 7, 2026
Merged

add function to preprocess for phase data#63
kenjiro-mk merged 9 commits intomasterfrom
dev_kuroda

Conversation

@KentoKuroda
Copy link
Collaborator

Phase dataを作成するためのコードを一通り作成したので、ご確認よろしくお願いいたします。

@KentoKuroda KentoKuroda requested a review from kenjiro-mk January 5, 2026 07:19
@kenjiro-mk
Copy link
Collaborator

@KentoKuroda
conflictが生じているようなので確認をお願いします.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.10

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.8

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.9

@KentoKuroda
Copy link
Collaborator Author

@kenjiro-mk お待たせしました。おそらく解消できたと思います。

@kenjiro-mk
Copy link
Collaborator

また,必要なライブラリなどは問題なかったでしょうか?もしあればpyproject.tomlに追加してください.そして,最後はprojectのversionを0.1.42のように1つアップデートをお願いします.このPRが終わればpypiでアップデートをする必要があります.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.10

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.8

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.9

@KentoKuroda
Copy link
Collaborator Author

新たに必要なライブラリはなかったためpyproject.tomlの内容は特に変えていませんが、バージョンのみ0.1.42に変更しました。お願いします。

@kenjiro-mk
Copy link
Collaborator

sb_event_path=f'D:/lab/My_Research/Github/OpenSTARLab/PreProcessing/test/sports/event_data/statsbomb/{sb_match_id}_events.pkl'

上記のような箇所を以下を参考に修正をお願いします.
.SARはテストデータを置いているのですが,それがなければハードコードされたパスをdatafactory_directory_pathのようなパスに変更してください.よろしくお願いいたします.

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.10

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.8

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Init Test Results 📝

  • Status: success
  • OS: Linux
  • Python: 3.9

@KentoKuroda
Copy link
Collaborator Author

ローカルのパスを除きました。また、テスト用のコードをコメントアウトしました。
ご確認よろしくお願いいたします。

elif data_provider == 'statsbomb_skillcorner':
sb_match_id = 3894537 # 843, 537
sc_match_id = 1018887 # 1498966, 1018887
sb_event_path=f'D:/lab/My_Research/Github/OpenSTARLab/PreProcessing/test/sports/event_data/statsbomb/{sb_match_id}_events.pkl'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.SARはテストデータを置いているのですが,それがなければハードコードされたパスをdatafactory_directory_pathのようなパスに変更してください.よろしくお願いいたします.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KentoKuroda
こちらも修正をお願いいたします.

@kenjiro-mk kenjiro-mk merged commit 1257de8 into master Jan 7, 2026
3 checks passed
@kenjiro-mk kenjiro-mk deleted the dev_kuroda branch January 7, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants